Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow non-serial port device addresses #209

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

olivierdelree
Copy link
Contributor

This PR brings the ability to supply custom addresses for the input and reward devices through the config.
These custom addresses are synced with the web interface so that they are available from the dropdown menu. Additionally, when starting a session, the address values are updated in the config.

This allows the config to dictate what ports are presented as options on
the main web interface.
The previous approach only listed serial ports
that were automatically detected. This commit provides the user with the
option of providing custom ports through the config, which is useful
when debugging on machines that do not have serial ports available.
@olivierdelree olivierdelree requested a review from celefthe May 29, 2024 16:47
@olivierdelree olivierdelree self-assigned this May 29, 2024
@olivierdelree olivierdelree added enhancement New feature or request interface Issue relates to the web interface labels May 29, 2024
@olivierdelree
Copy link
Contributor Author

Now that I look at it a bit more, maybe the defaults of /dev/ttyS0 and /dev/ttyS1 aren't the best choice as a Pi might prefer /dev/ttyUSB0 or something like that. Maybe the config should itself call serial so that it figures out default serial port from available?

@celefthe
Copy link
Member

@olivierdelree any ideas why the CI falls over?

Getting the following when the tests run

ERROR tests/test_launch.py - TypeError: Object of type SysFS is not JSON serializable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request interface Issue relates to the web interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Application does not allow custom input and reward addresses to be provided
2 participants